Skip to content

fix:events.js has a clerical error#9004

Closed
moreJs wants to merge 1 commit into
nodejs:masterfrom
moreJs:fix-event
Closed

fix:events.js has a clerical error#9004
moreJs wants to merge 1 commit into
nodejs:masterfrom
moreJs:fix-event

Conversation

@moreJs

@moreJs moreJs commented Oct 10, 2016

Copy link
Copy Markdown
Description of change

events.js has a clerical error

@nodejs-github-bot nodejs-github-bot added the events Issues and PRs related to the events subsystem / EventEmitter. label Oct 10, 2016
@bnoordhuis

Copy link
Copy Markdown
Member

'iff' is shorthand for 'if and only if'.

@moreJs

moreJs commented Oct 10, 2016

Copy link
Copy Markdown
Author

get it. thanks.

@claudiorodriguez

claudiorodriguez commented Oct 10, 2016

Copy link
Copy Markdown
Contributor

This does, however, beg the question - wouldn't it be clearer to just use if and only if?

EDIT: nevermind, iff is all over the codebase. I didn't think it was so widespread.

@bnoordhuis

Copy link
Copy Markdown
Member

It doesn't really matter here but if, say, you are documenting the preconditions of a complex algorithm, then 'iff' is less distracting than 'if and only if' spelled out ten times.

This seems to be the only place where 'iff' is used in the source. I'm fine with changing it but it's churn for very marginal benefits.

@cjihrig

cjihrig commented Oct 10, 2016

Copy link
Copy Markdown
Contributor

If we decide to move forward with this change, we should take the time to capitalize and punctuate the comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

events Issues and PRs related to the events subsystem / EventEmitter.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants